Draft
Conversation
ziopio
force-pushed
the
ziopio/pubsub
branch
from
fd342c5 to
6c7b533
Compare
|
Why is this not merged yet? |
Contributor
Author
|
IIRC, this PR implements the protocol but does not add anything to the informational model. It is just the protocol part of We left this here waiting to implement the full integration with the informational model and to make it useable by the user through OPCUA abstraction. Am I right @sylane ? Unfortunatly we never came back to it. |
Contributor
|
IIRC we had to stopped working on that, so there is some integration and testing work left. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A basic PubSub communication has been implemented but the way for it to read and write in the address space is hacky and should be disciplined before merging.
Things to do before merging this PR: